Skip to content

syz-manager: add Rank column and show the bug consequences as a tooltip #6271

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
tarasmadan merged 4 commits into from
Aug 28, 2025
Merged

syz-manager: add Rank column and show the bug consequences as a tooltip #6271

tarasmadan merged 4 commits into from
Aug 28, 2025

Conversation

@tarasmadan
Copy link
Collaborator

@tarasmadan tarasmadan commented Aug 11, 2025 •
edited
Loading

TODO:

  • Rank column - align right.
  • Default report links must trim the tail reports.
  • SYZFAIL must be ignored in tail reports.
  • Rank+Freq prefix must be fixed size.
  • Don't show tooltips where rank was not changed.
  • Make ranks with changes bold.

@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch 2 times, most recently from 1056e7c to cfbf564 Compare August 11, 2025 14:24
@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch 5 times, most recently from 371701d to 74d8cba Compare August 21, 2025 10:40
@tarasmadan
Copy link
Collaborator Author

tarasmadan commented Aug 25, 2025

Some preliminary result. Here the impact of bug was bumped from 8 to 17.
image

@tarasmadan
Copy link
Collaborator Author

tarasmadan commented Aug 26, 2025

After the UI improvements:

image

@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch 3 times, most recently from 7bdf2e8 to 7b8e099 Compare August 26, 2025 15:52
@tarasmadan tarasmadan marked this pull request as ready for review August 26, 2025 16:13
@tarasmadan tarasmadan requested a review from a-nogikh August 26, 2025 16:13
@tarasmadan tarasmadan changed the title vm: extract all the crashes from the log syz-manager: add Rank column and show the bug consequences as a tooltip Aug 26, 2025
@tarasmadan tarasmadan mentioned this pull request Aug 26, 2025
Open
4 tasks
@tarasmadan
Copy link
Collaborator Author

tarasmadan commented Aug 28, 2025

@a-nogikh ping

a-nogikh
a-nogikh previously approved these changes Aug 28, 2025
View reviewed changes
Copy link
Collaborator

@a-nogikh a-nogikh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch 2 times, most recently from 734666d to eeb7358 Compare August 28, 2025 09:17
@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch 2 times, most recently from 45d0b75 to 59323e4 Compare August 28, 2025 09:23
@tarasmadan
Copy link
Collaborator Author

tarasmadan commented Aug 28, 2025

@a-nogikh ptal

@tarasmadan tarasmadan requested a review from a-nogikh August 28, 2025 09:31
a-nogikh
a-nogikh reviewed Aug 28, 2025
View reviewed changes
@tarasmadan tarasmadan force-pushed the extract-multiple-reports branch from 59323e4 to 306f2e8 Compare August 28, 2025 10:17
@tarasmadan tarasmadan enabled auto-merge August 28, 2025 10:34
@tarasmadan tarasmadan added this pull request to the merge queue Aug 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 28, 2025
@tarasmadan tarasmadan added this pull request to the merge queue Aug 28, 2025
Merged via the queue into google:master with commit d401b9d Aug 28, 2025
31 of 32 checks passed
@tarasmadan tarasmadan mentioned this pull request Sep 3, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@a-nogikh a-nogikh a-nogikh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tarasmadan @a-nogikh